Skip to content

Conversation

hunterachieng
Copy link
Contributor

@hunterachieng hunterachieng commented Aug 14, 2025

Short Description

Add pdfShift documentation with examples.

Closes #679

AI Usage

Please disclose how you've used AI in this work (it's cool, we just want to
know!):

  • Code generation (copilot but not intellisense)
  • Learning or fact checking
  • Strategy / design
  • Optimisation / refactoring
  • Translation / spellchecking / doc gen
  • Other
  • I have not used AI

You can read more details in our
Responsible AI Policy

Copy link
Collaborator

@josephjclark josephjclark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The hardest bit of this, I think, is understanding how to use the PDF once it's been generated. What do you do with it?

I'm not sure you can pass it downstream to another step? Have we tried this? Because it may not serialise well, and that may cause problems either in the dataclip or in downstream state. And if so, that's stuff needs documenting.

Signed-off-by: Hunter Achieng <[email protected]>
@josephjclark
Copy link
Collaborator

@hunterachieng before we can merge this I think we need to look at OpenFn/adaptors#1338

And these docs are surfacing a couple of behavioural things that I'm worried about, like this JSON.parse step

I think we need to patch the adaptor before we release these docs

@hunterachieng
Copy link
Contributor Author

@josephjclark Since the JSON.parse comes from an option, should I add an example for that in the adaptor fix with the option and document it?

@josephjclark
Copy link
Collaborator

The option being filename?

What I really mean is: if PDF shift returns us a JSON response, we should automatically parse that and return it to state.data. That should be magic within the adaptor, not an intervention in job code

@hunterachieng
Copy link
Contributor Author

Understood. Let me work on that ASAP.

Signed-off-by: Hunter Achieng <[email protected]>
@josephjclark josephjclark merged commit 6830ca1 into main Aug 25, 2025
1 check passed
@josephjclark josephjclark deleted the feature/679-pdfShift-docs branch August 25, 2025 10:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Add docs for PDF generator adaptor
2 participants